-
Notifications
You must be signed in to change notification settings - Fork 169
feat: Small copy improvements #2189
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
TC-MO
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few thoughts to discuss but nothing that prevents merge. LGTM
sources/platform/index.mdx
Outdated
| **Apify** is a cloud platform and marketplace for web data extraction and automation tools called *Actors*. | ||
| > | ||
| > **Actors** are serverless cloud programs running on the Apify platform. They can perform simple actions like filling out web forms or sending emails, or complex operations like crawling millions of web pages or transforming large datasets. Actors can be started manually, via API, or on a schedule, and integrate easily with other applications. | ||
| > **Actors** are serverless cloud programs. They can perform simple actions like filling out web forms or sending emails, or complex operations like crawling millions of web pages or transforming large datasets. Actors can be started manually, via API, or on a schedule, and integrate easily with other applications. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few thoughts, I'd rather we get rid of the block quote, not 100% if it would be better as docusaurus admonition or maybe just as regular prose. Question is also do we even need Actor explanation here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated
| heading="Apify API" | ||
| description={<>Apify API provides programmatic access to the <Link to="/">Apify platform</Link></>} | ||
| heading="Apify API documentation" | ||
| description={<>Learn how to use the <Link to="/platform">Apify platform</Link> programmatically.</>} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wouldn't use Learn here since this is just the link to the full API ref, we don't teach there anything, and there is no narrative like for example in Academy's courses by @honzajavorek
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The current "Learn how to use" copy suggests guides or tutorials, but this page is reference documentation (REST API + client libraries).
I'd swap it out for something more developer-focused:
| description={<>Learn how to use the <Link to="/platform">Apify platform</Link> programmatically.</>} | |
| description={<>Integrate the <Link to="/platform">Apify platform</Link> into your applications using the API or client libraries.</>} |
It's developer-focused, outcome-oriented, and accurately cover both the REST API and client libraries covered on this page.
Other options if you prefer:
More general capability focus:
| description={<>Learn how to use the <Link to="/platform">Apify platform</Link> programmatically.</>} | |
| description={<>Access the <Link to="/platform">Apify platform</Link> programmatically using the API or client libraries.</>} |
Most concise (closest to original):
| description={<>Learn how to use the <Link to="/platform">Apify platform</Link> programmatically.</>} | |
| description={<>Access the <Link to="/platform">Apify platform</Link> programmatically.</>} |
I'd go with the "Integrate" version, but all three fix the main issue. Thoughts?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But that sounds like some CTA in Console. Whenever we link to docs from Console or other places, we often call the link "Learn more". IMO "learn" is perfectly okay
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah I understand the concern, and I appreciate you bringing those examples.
The key difference: those Stripe and Vercel examples link to content that has substantial how-to guides alongside the API reference.
This API landing page links to TWO different content types:
- Client libraries - some guided content, but primarily comprehensive API reference (classes, methods, interfaces)
- REST API - pure auto-generated OpenAPI spec (endpoint lists, parameters, responses)
Even our own client library docs (docs.apify.com/api/client/js and /python) don't use "Learn" in their descriptions - they describe what the library is rather than what you'll learn. The main selling point across all these sections is the referential documentation, with some guided content as supporting material.
Using "Learn" sets an expectation for tutorial-style instruction, but users primarily come here to look up API endpoints, method signatures, and parameters. If you are not keen on CTA-coded Integrate perhaps:
| description={<>Learn how to use the <Link to="/platform">Apify platform</Link> programmatically.</>} | |
| description={<>Access the <Link to="/platform">Apify platform</Link> programmatically using the API or client libraries.</>} |
would more accurately represent the reference-heavy nature of the content.
Happy to stick with Learn if you prefer, but wanted to flag that the current structure is more reference-focused than those comparison examples.
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
TC-MO
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It should have been approved in my first message, apologies
TC-MO
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fix typos
Co-authored-by: Michał Olender <92638966+TC-MO@users.noreply.github.com>
|
Preview for this PR was built for commit |
Co-authored-by: Michał Olender <92638966+TC-MO@users.noreply.github.com>
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |

Note
Docs copy and labeling updates
Apify platform documentationand refine Actor definition/intro copy.Apify API documentation, shorten SDK tab labels toJavaScript/Python, and rename client reference links toFull reference; fix TypeScript capitalization and misc phrasing.Apify SDK for JavaScript/Python, adjust descriptions, and rename SDK reference links toFull reference.UI/CSS
.ImageWrapper { overflow: hidden; }rule from API related articles grid.Written by Cursor Bugbot for commit a9bb224. Configure here.